Skip to content

fix(clients): honor project default models in new threads - #7

Closed
anirudhsama wants to merge 6 commits into
mainfrom
t3code/investigate-default-model-settings
Closed

fix(clients): honor project default models in new threads#7
anirudhsama wants to merge 6 commits into
mainfrom
t3code/investigate-default-model-settings

Conversation

@anirudhsama

Copy link
Copy Markdown
Owner

Review mirror of pingdotgg#6011, rebased onto current main with the two review findings addressed:

  • Web (zucram's comment): COMPOSER_DRAFT_STORAGE_VERSION bumped 8→9; migration strips modelSelectionByProvider/activeProvider from persisted drafts with no user content so stale empty drafts can't beat a project default.
  • Mobile (Cursor Bugbot's comment): decodePersistedComposerState strips bare modelSelection from new-task:* drafts left by the old post-send path; deliberately configured drafts are untouched.
  • Rebase conflict resolution keeps main's flushComposerDrafts, hydration-gated debounced writes, and atomic file writes.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L labels Aug 22, 2026
@anirudhsama
anirudhsama force-pushed the t3code/investigate-default-model-settings branch 2 times, most recently from 6d3c1dd to 5500e58 Compare August 22, 2026 13:46
@anirudhsama
anirudhsama force-pushed the t3code/investigate-default-model-settings branch from 5500e58 to 946268c Compare August 22, 2026 15:20
@github-actions

github-actions Bot commented Aug 24, 2026

Copy link
Copy Markdown

Thread transfer impact

⚠️ The latest CI run did not produce a thread transfer result for d7a5d31.

This comment will update automatically after the next completed run.

@anirudhsama

Copy link
Copy Markdown
Owner Author

Merged upstream

@anirudhsama anirudhsama closed this Sep 3, 2026
@anirudhsama
anirudhsama deleted the t3code/investigate-default-model-settings branch September 3, 2026 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants